Skip to content

adjust logging around cluster job#112

Merged
piceri merged 3 commits into
mainfrom
improve-cluster-job-logging
Jul 24, 2026
Merged

adjust logging around cluster job#112
piceri merged 3 commits into
mainfrom
improve-cluster-job-logging

Conversation

@piceri

@piceri piceri commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

adjust logging around cluster job

  • add logs to inform on status of a cluster job
  • set "Multiple digests observed" logs from Info to Debug to reduce noise

Signed-off-by: Eric Pickard <piceri@github.com>
@piceri
piceri requested a review from a team as a code owner July 24, 2026 16:22
Copilot AI review requested due to automatic review settings July 24, 2026 16:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves visibility into cluster-job processing while reducing rollout digest log noise.

Changes:

  • Adds cluster-job creation and polling logs.
  • Adds sync-record creation logging.
  • Moves multiple-digest messages from Info to Debug.
Show a summary per file
File Description
pkg/deploymentrecord/client.go Adds cluster-job lifecycle logs.
internal/controller/reporting.go Adjusts sync and digest-selection logging.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Medium

Comment thread pkg/deploymentrecord/client.go Outdated
piceri added 2 commits July 24, 2026 12:31
Signed-off-by: Eric Pickard <piceri@github.com>
Signed-off-by: Eric Pickard <piceri@github.com>
@piceri
piceri merged commit a125af9 into main Jul 24, 2026
8 checks passed
@piceri
piceri deleted the improve-cluster-job-logging branch July 24, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants